Skip to content

Post weekly search-report success digest to #engr#121

Merged
jpr5 merged 1 commit into
mainfrom
weekly-report-engr-success-ping
Jun 22, 2026
Merged

Post weekly search-report success digest to #engr#121
jpr5 merged 1 commit into
mainfrom
weekly-report-engr-success-ping

Conversation

@jpr5

@jpr5 jpr5 commented Jun 22, 2026

Copy link
Copy Markdown
Contributor

Summary

Follow-up to #120. The weekly search report previously only emitted to Slack on failure (#oss-alerts). A successful report is valuable signal, so on a successful Notion publish it now posts a digest to #engr:

📊 Pathfinder weekly search report — 2551 tool calls · 198 unique IPs · top: Agents/CoAgents/AG-UI · 3.3% empty · [report] (where [report] is a Slack mrkdwn link to the Notion page).

  • New SLACK_ENGR_WEBHOOK env (workflow secret SLACK_WEBHOOK_ENGR, a B0T #engr incoming webhook — same convention as SLACK_WEBHOOK_OSS_ALERTS/SLACK_WEBHOOK_TEAM_SHOWCASE).
  • Failures unchanged — still fail-loud to #oss-alerts via SLACK_WEBHOOK.
  • Best-effort success ping: no-ops when the webhook is unset (so this is safe to merge before the secret exists) and never affects the run's exit code.

Test plan

  • Red-green: success run posts the digest (incl. <url|report>) to the success poster and NOT the failure poster; fetch/token failure posts to the failure poster and NOT the success poster; unset success webhook no-ops. (52 report tests; full suite 6522 green)
  • tsc (root + scripts), npm run build, prettier --check, actionlint clean
  • After merge: create the B0T #engr incoming webhook → set org secret SLACK_WEBHOOK_ENGR; next Sunday run (or a workflow_dispatch) posts to #engr

7-agent CR converged on round 1 (zero in-subject findings); pre-existing report-rendering follow-ups (table backslash-escape, categorization useCoAgent keyword, parseReportDays cap, etc.) tracked from #120, not in scope here.

🤖 Generated with Claude Code

success → #engr (SLACK_WEBHOOK_ENGR) with a digest + [report] link; failures
unchanged on #oss-alerts; no-op if the success webhook is unset.
@jpr5 jpr5 merged commit e015d69 into main Jun 22, 2026
9 checks passed
@jpr5 jpr5 deleted the weekly-report-engr-success-ping branch June 22, 2026 23:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant